Skip to content

replace universal quantifier with existential in knitr compatibility claim #1712

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: prerelease
Choose a base branch
from

Conversation

cscheid
Copy link
Collaborator

@cscheid cscheid commented Jul 16, 2025

For consistency with "highly" (vs "perfectly"), "generally" (vs "always"), and "should" (vs "must"), let's use "an" (vs "any").

@github-actions github-actions bot temporarily deployed to pull request July 16, 2025 13:36 Inactive
Copy link
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1712.quarto.org

🔄 Modified Documents

Copy link
Collaborator

@cderv cderv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can also do the update on main for current website ?

It doesn't seem tied to prerelease version.

@cscheid
Copy link
Collaborator Author

cscheid commented Jul 16, 2025

Sure - this isn't high priority and would just get eventually merged to main, but we can do an extra PR there.

@cderv
Copy link
Collaborator

cderv commented Jul 17, 2025

would just get eventually merged to main

This PR is against the prerelease branch, so I was thinking we could do it to main instead, and then we would have auto-commit to prerelease with the automated PR that would be created.

So usually, when something can be fixed or have more precision, then I do it in main through PR to get auto port to prerelease branch. That way users benefits right now for the improved doc.

Both ways are good - just wanted to share this practice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants